Skip to content

Conversation

mariehd
Copy link
Member

@mariehd mariehd commented Sep 4, 2026

Endringer

  • Opprettet SongList-komponent
  • Viser alle sanger fra sanglisten
  • Hver sang kan velges med en knapp
  • La til onSelectSong som callback-prop
  • Callbacken sender valgt Song tilbake til komponentens forelder
  • Bruker sangens id som React key

Test

  • npm run lint fungerer uten feil
  • npm run build fungerer uten feil
  • Alle sangene vises i oversikten når man kjører appen med npm run dev etter du har byttet ut innholdet (midlertidig) i app.tsx med dette:
import { Navigation } from './components/Navigation'
import { SongCard } from './components/SongCard'
import { SongList } from './components/SongList'
import { songs } from './data/songs'

function App() {
  const handleSelectSong = (song: (typeof songs)[number]) => {
    console.log('Valgt sang:', song)
  }

  return (
    <main>
      <SongCard song={songs[0]} />
      <Navigation isFirst={true} isLast={false} />
      <SongList songs={songs} onSelectSong={handleSelectSong} />
    </main>
  )
}

export default App
  • Åpne Developer Tools og så Console. Klikk på en sang, dette vil trigge callbacken og viser valgt sang i konsollen
  • Fjern endringen i app.tsx igjen, tilbake til det som var der fra før

Closes #10

@mariehd mariehd requested a review from a team as a code owner September 4, 2026 07:49
@mariehd mariehd linked an issue Sep 4, 2026 that may be closed by this pull request
2 tasks
ellahw
ellahw previously approved these changes Sep 4, 2026
Copy link
Member

@ellahw ellahw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Se kommentar i koden. Bare en liten endring, så ser alt bra ut!

Copy link
Member

@ellahw ellahw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bra!!

@ellahw ellahw merged commit 0cb09c4 into main Sep 4, 2026
@ellahw ellahw deleted the 10-lag-oversikthopp-til-sang-komponent-statisk branch September 4, 2026 15:06
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lag songList-komponent, statisk
2 participants